Skip to content

Add Flipper TV Remote#1024

Merged
xMasterX merged 5 commits into
flipperdevices:mainfrom
jmanion0139:jmanion0139/flipper_tv_remote_1.0
Jun 8, 2026
Merged

Add Flipper TV Remote#1024
xMasterX merged 5 commits into
flipperdevices:mainfrom
jmanion0139:jmanion0139/flipper_tv_remote_1.0

Conversation

@jmanion0139

@jmanion0139 jmanion0139 commented Apr 19, 2026 •
edited by xMasterX
Loading

Copy link
Copy Markdown
Contributor

Application Submission

  • An infrared application which is designed to replace a TV remote. Instead of navigating menus to select what signal to send, 12 signals can be recorded and sent from short or long pressing the D-pad and Back buttons. This allows it to act fully as a TV remote with ease.

Extra Requirements

  • None

Author Checklist (Fill this out)

  • I've read the contribution guidelines and my PR follows them
  • I own the code I'm submitting or have code owner's permission to submit it
  • I have validated the manifest file(s) with python3 tools/bundle.py --nolint applications/CATEGORY/APPID/manifest.yml bundle.zip

Reviewer Checklist (Don't fill this out)

  • Bundle is valid
  • There are no obvious issues with the source code
  • I've ran this application and verified its functionality

@jmanion0139 jmanion0139 changed the title jmanion0139/flipper tv remote 1.0 Add Flipper TV Remote Apr 24, 2026
xMasterX
xMasterX requested changes May 23, 2026
View reviewed changes

@xMasterX xMasterX left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello, app should not store its config files in other app folders like you did in
TV_REMOTE_SETTINGS_PATH

Please use apps_data folder for app related config files

@xMasterX xMasterX added the fixes needed Unresolved issues with the app requiring maintainer's attention label May 23, 2026
@jmanion0139

jmanion0139 commented May 27, 2026

Copy link
Copy Markdown
Contributor Author

Hello, thank you for letting me know about that. Should this apply to both config files for my application, as well as the infrared remote files it creates when recording a remote, or can those files stay getting saved to ANY_PATH("infrared") as they are compatible with the built-in infrared app?

Thank you

@jmanion0139 jmanion0139 requested a review from xMasterX May 30, 2026 18:43
@xMasterX

xMasterX commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

Hello, thank you for letting me know about that. Should this apply to both config files for my application, as well as the infrared remote files it creates when recording a remote, or can those files stay getting saved to ANY_PATH("infrared") as they are compatible with the built-in infrared app?

Thank you

If they are compatible they can stay in infrared folder, but better to separate apps files and "main apps" files since it may lead to some confusion

@xMasterX xMasterX removed the fixes needed Unresolved issues with the app requiring maintainer's attention label Jun 8, 2026
@xMasterX xMasterX merged commit f246699 into flipperdevices:main Jun 8, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xMasterX xMasterX xMasterX approved these changes

Assignees

No one assigned

Labels

app (new)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jmanion0139 @xMasterX